SPT-1998 Обновление проекта с примерами #130
Conversation
mrandrewsmith
requested review from
chausovSurfStudio,
NullIsOne and
Goldilocks97
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 5.0.0 #130 +/- ##
==========================================
- Coverage 91.42% 91.39% -0.03%
==========================================
Files 82 83 +1
Lines 1259 1267 +8
==========================================
+ Hits 1151 1158 +7
- Misses 108 109 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
mrandrewsmith
force-pushed
the
SPT-1998-make-mock-target
branch
from
c927a83 to
babfb15
Compare
mrandrewsmith
force-pushed
the
SPT-1998-example-refresh
branch
from
75b8324 to
3bc68ee
Compare
NodeKit/NodeKitTests/UnitTests/AsyncIterator/AsyncPagerIteratorTests.swift
Outdated
Show resolved
Hide resolved
Example/Example/Features/PaginationFeature/Cells/PaginationCell.swift
Outdated
Show resolved
Hide resolved
Example/Example/Features/PaginationFeature/Cells/PaginationCellGenerator.swift
Outdated
Show resolved
Hide resolved
Example/Example/Features/PaginationFeature/Subviews/LoadingView.swift
Outdated
Show resolved
Hide resolved
Example/Example/Flows/FeatureListFlow/Cells/FeatureCellGenerator.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
еще есть такое предложение - немного изменить структуру
сейчас что немного покоробило: есть Example-проект, есть локальные зависимости, есть сторонние зависимости. При этом локальные разделяются на непосредственно саму либу (NodeKit и его ThirdParty), и на "модули" самого проекта. Но все это лежит вместе в одной кучке(
что если часть из них, которые непосредственно "модули" самого Example-проекта (MockServer, Services, Models), внести в проект - типа вот как тут
ну чтобы явно было видно, что вот модули проекта - да, они в отдельном пакете, но это все еще example, можно его менять, в вашем проекте в целом будет примерно так же. А все что ниже - это уже сторонние либы, да, часть из них локальные, но представьте что это черная коробка и не обращайте внимания на них.
Еще - возможно было бы полезно как-то иметь возможность изменять интервал, через который приходят ответы от мокового сервака. Я пока код не посмотрел - так и не понял, где там пагинация и как она работает, все моментально выполняется)
В остальном, помню какой тут бардак был раньше - и сейчас прям симпатично стало, работа оч большая проведена 💪
Example/Services/Services/PaginationService/PaginationContentDataProvider.swift
Outdated
Show resolved
Hide resolved
NodeKit/NodeKitTests/UnitTests/AsyncIterator/AsyncPagerIteratorTests.swift
Outdated
Show resolved
Hide resolved
mrandrewsmith
force-pushed
the
SPT-1998-make-mock-target
branch
from
babfb15 to
77a98d8
Compare
mrandrewsmith
force-pushed
the
SPT-1998-example-refresh
branch
from
5804141 to
da998b5
Compare
Что сделано
Как проверить